Skip to content

Added archived field to Task model#1019

Merged
joshsmith merged 1 commit intocode-corps:developfrom
vishaldeepak:1006-add-archieived-to-task
Oct 8, 2017
Merged

Added archived field to Task model#1019
joshsmith merged 1 commit intocode-corps:developfrom
vishaldeepak:1006-add-archieived-to-task

Conversation

@vishaldeepak
Copy link
Copy Markdown
Contributor

@vishaldeepak vishaldeepak commented Oct 6, 2017

What's in this PR?

Added Archived field to Tasks

Fixes #1006

@joshsmith
Copy link
Copy Markdown
Contributor

Hey @vishaldeepak I know this wasn't in the task, but can you add this to the TaskView and TaskViewTest so we can be sure it's emitted to the front-end? Make sure they're alpha-ordered.

Copy link
Copy Markdown
Contributor

@joshsmith joshsmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick change, and see comment above.

Comment thread lib/code_corps/model/task.ex Outdated
struct
|> changeset(params)
|> cast(params, [:status])
|> cast(params, [:status, :archived])
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you reorder this to be in alpha order?

%{archived: archived} = changeset.changes
assert changeset.valid?
assert archived
end
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solid test here. 👍

@vishaldeepak vishaldeepak force-pushed the 1006-add-archieived-to-task branch 2 times, most recently from b4ced41 to df742d4 Compare October 8, 2017 03:38
@vishaldeepak
Copy link
Copy Markdown
Contributor Author

Made changes accordingly

@joshsmith joshsmith force-pushed the 1006-add-archieived-to-task branch from df742d4 to ab28deb Compare October 8, 2017 15:13
joshsmith
joshsmith previously approved these changes Oct 8, 2017
@joshsmith joshsmith force-pushed the 1006-add-archieived-to-task branch from ab28deb to 2cc067f Compare October 8, 2017 15:16
joshsmith
joshsmith previously approved these changes Oct 8, 2017
@joshsmith joshsmith force-pushed the 1006-add-archieived-to-task branch from 2cc067f to 491e218 Compare October 8, 2017 15:20
@joshsmith joshsmith merged commit 199d2b5 into code-corps:develop Oct 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants